До Compare Objects додались ще два аргументи, які не були передбачені, але стануть в нагоді.

Зараз з цим PR я бачу наступне:

------------------------------------------------------------------------------
Неможливість змінити початкову вартість тендера                       | FAIL |
Expected error 'Failed to modify "value.amount"' but got 'Calling method 'patch_tender' failed: RequestFailed: Internal Server Error'.
------------------------------------------------------------------------------
Неможливість змінити мінімальний крок тендера                         | FAIL |
Expected error 'Failed to modify "minimalStep.amount"' but got 'Calling method 'patch_tender' failed: RequestFailed: Internal Server Error'.
------------------------------------------------------------------------------

------------------------------------------------------------------------------
Можливість внести зміни у тендер після запитання                      | FAIL |
Failed to modify "description"
------------------------------------------------------------------------------

+(review wanted) +@kosaniak +@annawzz

Є і суттєвий недолік в такому підході: всі майданчики теж мусять видавати помилку з таким повідомленням коли в них провалюється ключове слово на зміну того чи іншого поля. Тому... +(backward incompatible)

Було б дуже добре, якби в драйверах майданчиків теж можна було розрізняти деякі конкретні помилки.

Докинув коміт, який переміщає артефакт в ${OUTPUT_DIR}. Це доволяє запускати багато екземплярів тестів паралельно і повністю незалежно з допомогою одної лиш опції -d (--outputdir).

Reviewed 1 of 1 files at r1, 1 of 1 files at r2, 1 of 1 files at r3, 1 of 1 files at r4, 1 of 1 files at r5, 5 of 5 files at r6, 2 of 2 files at r7, 2 of 2 files at r8.
Review status: all files reviewed at latest revision, 5 unresolved discussions.

op_robot_tests/tests_files/service_keywords.py, line 437 at r1 (raw file):

        if element_id == object_id:
            break
        else:

Дарма ти це зробив. else стосувалося for (вихід з циклу без break). Тобто, якщо такої айдішки не знаходилося, то це якийсь новий елемент і ми його дані ще не зчитували. Тоді ми логічно припускаємо, що його індекс буде на одиницю більшим, ніж останнього прочитаного елементу.

op_robot_tests/tests_files/service_keywords.py, line 434 at r3 (raw file):

    if 'title' in obj.keys():
        title_id = _get_id_from_object(obj, 'title')

FYI: це можна написати трошки по-іншому

title_id = _get_id_from_object(obj, 'title') if 'title' in obj.keys() else None

Чи так краще - вирішувати тобі.

op_robot_tests/tests_files/service_keywords.py, line 427 at r4 (raw file):

                mandatory key, either "title" or "description".
                The value mapped to that key should begin with a prefix which
                holds an automatically generated identifier of the object

Можеш ще вказати функцію, яка ту айдішку додає. Було б корисно.

op_robot_tests/tests_files/service_keywords.py, line 84 at r5 (raw file):
Якщо first буде не (юнікодовим???) стрінгом, то цей unicode нічого не дасть:

>>> a=u"aaaàçççñññ"
>>> print a
aaaàçççñññ
>>> print unicode(a)
aaaàçççñññ
>>> x = [1, a, 2, 3]
>>> print unicode(x)
[1, u'aaa\xe0\xe7\xe7\xe7\xf1\xf1\xf1', 2, 3]
>>> b = 'aaa\xe0\xe7\xe7\xe7\xf1\xf1\xf1'
>>> b
'aaa\xe0\xe7\xe7\xe7\xf1\xf1\xf1'
>>> unicode(b)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe0 in position 3: ordinal not in range(128)
>>> c = u'aaa\xe0\xe7\xe7\xe7\xf1\xf1\xf1'
>>> c
u'aaa\xe0\xe7\xe7\xe7\xf1\xf1\xf1'
>>> unicode(c)
u'aaa\xe0\xe7\xe7\xe7\xf1\xf1\xf1'
>>> print c
aaaàçççñññ

op_robot_tests/tests_files/brokers/openprocurement_client.robot, line 196 at r7 (raw file):

  # The two values should differ. If they are equal, then probably the server refused to,
  # or failed to modify the value of a field, so this keyword should instantly fail.
  ${new_value}=  Get From Object  ${USERS.users['${username}'].tender_data.data}  ${fieldname}

Це ж будуть старі дані, нє? Тре брати Get From Object ${tender.data}....

Comments from Reviewable

Класне рев'ю!

Reviewed 1 of 1 files at r2, 1 of 1 files at r3, 1 of 1 files at r5, 5 of 5 files at r6, 2 of 2 files at r7, 2 of 2 files at r8.
Review status: all files reviewed at latest revision, 5 unresolved discussions.

op_robot_tests/tests_files/service_keywords.py, line 437 at r1 (raw file):

op_robot_tests/tests_files/service_keywords.py, line 434 at r3 (raw file):

op_robot_tests/tests_files/service_keywords.py, line 427 at r4 (raw file):

op_robot_tests/tests_files/service_keywords.py, line 84 at r5 (raw file):

op_robot_tests/tests_files/brokers/openprocurement_client.robot, line 196 at r7 (raw file):

Comments from Reviewable

Дякую :)

Review status: all files reviewed at latest revision, 5 unresolved discussions.

op_robot_tests/tests_files/service_keywords.py, line 84 at r5 (raw file):

Comments from Reviewable

Review status: all files reviewed at latest revision, 5 unresolved discussions.

op_robot_tests/tests_files/service_keywords.py, line 84 at r5 (raw file):

Comments from Reviewable

Review status: all files reviewed at latest revision, 5 unresolved discussions.

op_robot_tests/tests_files/service_keywords.py, line 84 at r5 (raw file):

Comments from Reviewable

Review status: all files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

op_robot_tests/tests_files/service_keywords.py, line 84 at r5 (raw file):
Ну це воно і є. :)

>>> class C:
...   def __str__(self):
...     return 'nothing'
... 
>>> c = C()
>>> c
<__main__.C instance at 0x7fb507c8d290>
>>> repr(c)
'<__main__.C instance at 0x7fb507c8d290>'
>>> str(c)
'nothing'

Comments from Reviewable